Fixed as requested.

Anything else missing here? Would love to get this into main.

The tests are red because this doesn'te valuate on aarch64-linux anymore. So this feature should be made x86_64-linux only. That being said, I'll try to land #50 first, then we can ship this, but w/o the overhead in the closure for everyone else (I know that this wouldn't have happened with the BindReadOnlyPaths approach, but I still think that early patching paths in the build is the superior approach).

Force-pushed a commit that adds this to its own output.

Would've loved to assert on this, i.e.

buildGoModule {
  # ...
  __structuredAttrs = true;
  outputChecks.out = {
    disallowedRequisites = [ guacamole-server ];
  };
}

but apparently buildGoModule doesn't support structured attrs :(

@dminuoso just pushed a change that also turns the thing off and throws an assertion error if guacamole-server is not available (e.g. aarch64-linux). Please double-check if this is OK for you or if I did regress something (not using this feature myself, sorry).

@Ma27 Works fine on our servers. Can be merged.